Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1066 databricks support #1067

Draft
wants to merge 19 commits into
base: main
Choose a base branch
from
Draft

1066 databricks support #1067

wants to merge 19 commits into from

Conversation

benfdking
Copy link
Collaborator

  • temp working on things
  • feat: added all tests and started implemented dialect

@benfdking benfdking linked an issue Dec 17, 2024 that may be closed by this pull request
1 task
@benfdking benfdking force-pushed the 1066-databricks-support branch from 368cddd to 2de72cb Compare December 17, 2024 13:20
Copy link

Benchmark for bf75ae7

Click to view benchmark
Test Base PR %
DepthMap::from_parent 40.0±0.34µs 41.2±0.49µs +3.00%
fix_complex_query 10.5±0.11ms 10.4±0.11ms -0.95%
parse_complex_query 2.8±0.04µs 2.9±0.05µs +3.57%
parse_expression_recursion 5.1±0.04µs 5.1±0.06µs 0.00%
parse_simple_query 816.8±8.00ns 797.0±12.80ns -2.42%

Copy link

Benchmark for be5a27d

Click to view benchmark
Test Base PR %
DepthMap::from_parent 40.3±0.22µs 54.9±0.20µs +36.23%
fix_complex_query 10.2±0.20ms 10.3±0.10ms +0.98%
parse_complex_query 2.9±0.02µs 2.9±0.04µs 0.00%
parse_expression_recursion 5.1±0.06µs 5.1±0.06µs 0.00%
parse_simple_query 801.4±9.12ns 809.6±17.83ns +1.02%

@benfdking benfdking force-pushed the 1066-databricks-support branch 3 times, most recently from c6f9879 to 0c99158 Compare December 19, 2024 10:33
@benfdking benfdking force-pushed the 1066-databricks-support branch from 0c99158 to 0f09194 Compare December 21, 2024 12:40
Copy link

Benchmark for 976e455

Click to view benchmark
Test Base PR %
DepthMap::from_parent 40.6±0.35µs 39.6±0.54µs -2.46%
fix_complex_query 10.2±0.14ms 10.3±0.06ms +0.98%
parse_complex_query 2.8±0.03µs 2.9±0.05µs +3.57%
parse_expression_recursion 5.1±0.08µs 5.1±0.06µs 0.00%
parse_simple_query 816.4±23.93ns 797.7±4.59ns -2.29%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

databricks support
1 participant